-
Notifications
You must be signed in to change notification settings - Fork 6
Publish to PyPI on release #16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
release: | ||
types: | ||
- published |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needed for the new upload_all
job below, which has:
if: github.event_name == 'release' && github.event.action == 'published'
@@ -21,20 +25,38 @@ jobs: | |||
build_wheels: | |||
strategy: | |||
matrix: | |||
os: ["ubuntu-20.04", "macos-11"] | |||
os: ["ubuntu-latest", "macos-latest"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cibuildwheel
should take care of the compatibility for older platforms
Do we need to pass any secrets to Github for Pypi? |
Per this document, no - notice how the |
Nice. Let's merge and try publishing... |
Re: Issue #15, this PR updates the
build.yml
file such that:.tar.gz
) is created.whl
) are uploaded to PyPII am drawing from these docs:
cibuildwheel/examples/github-deploy.yml
I have created a GitHub "environment" called
release
for publishing. I will configure PyPI to use it as well.